Skip to content

chore(ci): remove redundant pip/setuptools/wheel upgrade step from wo… - #17715

Merged
parthea merged 4 commits into
mainfrom
feat/cleanup-import-profiler-workflow-only
Jul 15, 2026
Merged

chore(ci): remove redundant pip/setuptools/wheel upgrade step from wo…#17715
parthea merged 4 commits into
mainfrom
feat/cleanup-import-profiler-workflow-only

Conversation

@hebaalazzeh

Copy link
Copy Markdown
Contributor

This PR removes the redundant Upgrade pip, setuptools, and wheel step from the host runner environment in .github/workflows/import-profiler.yml.

Since ci/run_single_test.sh now initializes its own virtual environment and upgrades/installs pip and setuptools inside .venv-profiler, upgrading them in the host runner environment is redundant and has no effect on the actual profile checks.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@hebaalazzeh
hebaalazzeh marked this pull request as ready for review July 14, 2026 22:05
@hebaalazzeh
hebaalazzeh requested a review from a team as a code owner July 14, 2026 22:06
@parthea
parthea merged commit fcaad3c into main Jul 15, 2026
40 checks passed
@parthea
parthea deleted the feat/cleanup-import-profiler-workflow-only branch July 15, 2026 14:43
hebaalazzeh added a commit that referenced this pull request Jul 15, 2026
#17715)

This PR removes the redundant `Upgrade pip, setuptools, and wheel` step
from the host runner environment in
`.github/workflows/import-profiler.yml`.

Since `ci/run_single_test.sh` now initializes its own virtual
environment and upgrades/installs `pip` and `setuptools` inside
`.venv-profiler`, upgrading them in the host runner environment is
redundant and has no effect on the actual profile checks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants